Skip to content

Conversation

ceyonur
Copy link
Collaborator

@ceyonur ceyonur commented Oct 2, 2025

Why this should be merged

ACP-226 directly affects the block building time. This PR adds ACP-226 delays to block builder.

How this works

It adds 2 delay system to the block builder:
1- ACP-226 based remaining min delay after parent's time. This is in order to pass the verification (to be added).
2- A constant min retry delay

How this was tested

added UTs

Need to be documented?

no

Need to update RELEASES.md?

updated

@ceyonur ceyonur changed the title Ceyonur/acp 226 min block delay builder ACP-226: Apply Min Block Delay to Block Builder Oct 2, 2025
Base automatically changed from ceyonur/wait-for-event-refactor to master October 2, 2025 18:13
@ceyonur ceyonur marked this pull request as ready for review October 3, 2025 13:14
@ceyonur ceyonur requested a review from a team as a code owner October 3, 2025 13:14
@ceyonur ceyonur force-pushed the ceyonur/acp-226-min-block-delay-builder branch from f02e5c7 to aa9a2cc Compare October 3, 2025 13:16
@ceyonur ceyonur self-assigned this Oct 3, 2025
@ceyonur ceyonur linked an issue Oct 3, 2025 that may be closed by this pull request
Copy link
Contributor

@StephenButtolph StephenButtolph left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

Copy link
Contributor

@michaelkaplan13 michaelkaplan13 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One comment suggestion, but LGTM

@ceyonur ceyonur enabled auto-merge October 8, 2025 07:53
@ceyonur ceyonur added this pull request to the merge queue Oct 8, 2025
Merged via the queue into master with commit e49a625 Oct 8, 2025
9 checks passed
@ceyonur ceyonur deleted the ceyonur/acp-226-min-block-delay-builder branch October 8, 2025 08:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

handle/enforce min block delay

4 participants